Clang immintrin bug - Githubissues

luxonis / XLink

A cross-platform library for communicating with devices over various physical links.

Apache License 2.0

11 stars 17 forks source link

Clang immintrin bug #80

Open onthegrid007 opened 2 weeks ago

onthegrid007 commented 2 weeks ago

HI I have been trying to deduce this bug from the depthai-core library and found out that it is a simple fix but do not know how to make a pull request...

In the file: XLink/src/pc/protocols/usb_host.cpp

The std headers just need to be moved to the top of the file for this bug to be fixed... I assumed it was a clang bug but it turns out its an order of header bug.

This would make the xlink library clang compatible if this single issue is fixed

themarpe commented 2 weeks ago

@onthegrid007 you may post the diff here and I can apply the fix Otherwise WRT making a PR, you can:

Fork the XLink repository
commit the change that fixes it to your fork
click New Pull Request under Luxonis XLink repo and select your fork/master and our master branch

onthegrid007 commented 2 weeks ago

I will follow this this weekend, middle of school atm