[PATCH] D47553: Add TargetLowering::shouldExpandAtomicToLibCall and query it from AtomicExpandPass

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 31 09:07:30 PDT 2018


jfb added a comment.

In https://reviews.llvm.org/D47553#1117633, @asb wrote:

> Updated patch to document that shouldExpandAtomicToLibCall should return the same result for objects of the same size.
>
> @jfb: I'm simply replicating the existing logic with regards to checking alignment - no functional change is intended.


I see! Can you refer to the langref, which says this is UB, and say you're trying your best to not be a jerk about it? Or something like that :-)


https://reviews.llvm.org/D47553





More information about the llvm-commits mailing list