dblaikie added a comment.

In D67647#1673363 <https://reviews.llvm.org/D67647#1673363>, @comex wrote:
> Ugh, it looks like `getArgs()` is a massive strict aliasing violation.  And 
> it's used in enough different places in Clang that fixing the violation may 
> require extensive refactoring... I've reported the issue as 
> https://bugs.llvm.org/show_bug.cgi?id=43344, but I don't really want to fix 
> it :)
>
> However, I can at least update this patch to avoid using getArgs().


Yeah, existing technical debt - I don't think your situation is making that any 
worse or better, so no worries about it.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67647/new/

https://reviews.llvm.org/D67647



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to