Context: https://reviews.llvm.org/D120380

Capture options.sectionMemoryMapper by value instead of by reference

In D112736#3094954, @silvas wrote:

1. I would just like Math dialect transforms to not depend on target-specific dialects. So concretely just RsqrtOpApproximation pattern would move to x86vector/Transform/, since it is the target specific one.
2. I agree, removing Polynomial would be good.

In D112729#3094170, @ezhulenev wrote:

Included this fix in: https://reviews.llvm.org/D112736

Thanks for this! Just some nits below, feel free to ignore.

• Re-indented test/Dialect/Math/polynomial-approximation-avx2.mlir as suggested by pifon2a.

• Remove stray comments from a previous iteration
• Rename flag from avx2 to enableAvx2/enable-avx2
• Document the enableAvx2 bool in the Options struct
• Use FileCheck --check-prefix; remove the now-redundant _avx2.mlir file
• Fix coding style issues

Add missing newline's at EOFs

After discussing with Mehdi and Aart, I'm withdrawing this change in favor of https://reviews.llvm.org/D111461

Thanks Tobias for the quick reply. Can you please commit this change? I don't yet have commit access to the repo.

Rebase onto HEAD

Fix file header + trailing newline

Ping.
I just added log1p's approximation (https://reviews.llvm.org/D101765). I'd like this patch to go in so that I can also add expm1's approximation.

mention inf in the comment

In D100344#2686777, @mehdi_amini wrote:

The ORC bug is closed, can you update all these tests back to use ORC?

• Use std.constant dense
• Mention https://bugs.llvm.org/show_bug.cgi?id=49906 in test-rsqrt.mlir

In D100344#2684413, @mehdi_amini wrote:

Ideally the bug should be referred as TODO in all these tests, and also these tests should be mentioned in the bug so that when closing the bug we update the tests back.

Add TODO comments that refer to the reported bug.

In D100344#2684260, @aartbik wrote:

aren't the AMX test affected by this as well then?

Fix AMX tests as well

fix newline

Rebase on X86Vector

So far I only have immediate plans of adding avx.rsqrt to the dialect. I'd expect AVX, AVX512 and SSE ops to be added to this dialect in the future.

Fix overly long comments

In D100119#2678531, @bondhugula wrote:

There isn't a commit summary at all. It'd be good to explain the rationale for the renaming.

The previous push used a different diffbase than the original push,
which messed up the diff. Push a diff with the original diffbase now.

• [mlir] Rename AVX512 dialect to X86Vector

Context: https://reviews.llvm.org/D99818

Address some reviewer comments

• Fix typo in comment.

• Test fixes as suggested by mehdi_amini.
• Log base class as suggested by ezhulenev.

s/LOG_OP/Op/

Update D98282: [mlir] Add polynomial approximation for math::Log2

I have no commit access -- please merge it, thanks!