[PATCH] D86021: [IR] Make nosync, nofree and willreturn default for intrinsics.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 30 07:15:31 PDT 2020


nikic added a comment.

I find the naming here confusing: Looking at this patch my expectation was that `DefaultIntrinsic` is used for intrinsics with default attributes, based on the name. Took me a while to understand it's actually the other way around. Shouldn't the Intrinsic <-> DefaultIntrinsic naming be inverted?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86021



More information about the llvm-commits mailing list